[FIX]: Fails for parallel test - #49

I think these changes are already a part of ongoing test-revamp in this branch: https://github.com/compiler-research/cppjit/tree/test-suite-infra, but we can land this independently. Thanks.

Can you explain how this change resolved the parallelism problems?

Can you explain how this change resolved the parallelism problems?

test09_templated_function (TestTUTORIALFEATURES) relied on another class (TestDOCFEATURES) to cppdef multiply introducing an inter-test setup dependency.

test03_memory never imported ll itself making the module unreachable unless the same worker happened to have an earlier test that did so.

This change was already tested with 87e799c

Can you explain how this change resolved the parallelism problems?

The first change was just a definition replacement, I just moved the definition of multiply function, where it is actually used.

In the second I just added import ll, the test was using ll but was not importing it, so it was dependent on previous tests' import, so it was failing sometimes in parallel mode

Ok, thanks -- in retrospect, probably worth being part of the commit message.